Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[braket] support reservation ARN #2553

Merged
merged 7 commits into from
Feb 8, 2025
Merged

Conversation

yitchen-tim
Copy link
Contributor

Description

Add support to reservation ARN. When AMZN_BRAKET_RESERVATION_TIME_WINDOW_ARN is presented in the environment variable (either from hybrid jobs with reservation or added manually), the reservation arn is added as part of create quantum task.

Copy link

copy-pr-bot bot commented Jan 28, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Copy link
Collaborator

@khalatepradnya khalatepradnya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank, Tim.

@khalatepradnya
Copy link
Collaborator

khalatepradnya commented Feb 7, 2025

Please format the updated files. clang-format-16 -i <file> or run the formatting script bash scripts/run_clang_format.sh

@yitchen-tim
Copy link
Contributor Author

@khalatepradnya thanks, I have formatted the code

@khalatepradnya
Copy link
Collaborator

khalatepradnya commented Feb 8, 2025

/ok to test

Command Bot: Processing...

@khalatepradnya khalatepradnya enabled auto-merge (squash) February 8, 2025 00:06
@khalatepradnya khalatepradnya merged commit 20e5869 into NVIDIA:main Feb 8, 2025
212 of 213 checks passed
github-actions bot pushed a commit that referenced this pull request Feb 8, 2025
Copy link

github-actions bot commented Feb 8, 2025

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

github-actions bot pushed a commit that referenced this pull request Feb 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants